Also, it doesn't look like this removes the whole submodule?

Also, it doesn't look like this removes the whole submodule?

true i am doing it in phases
it helps me keep track and easily spot failures.
next is resources
just having power issues over

Ah ok! Do you mind if I convert this to a draft then? (I like the idea of doing this in stages!)

Ah ok! Do you mind if I convert this to a draft then? (I like the idea of doing this in stages!)

It's fine with
Should I continue to push more commits on this same pr or should i make separate ones for each commit?

Let's keep it all in one PR, if that's alright...

Let's keep it all in one PR, if that's alright...

Alright then
Thank you @ibacher

hello @ibacher
here is an update on this ticket , i have managed to migrate the resource classes to from omod 1.8 to 1.9
however i ran into trouble with some test classes which cause test failures;
following up on the talk thread Testcase failures

1. on PersonAttributeResource1_9Test() should we ignore the setValue_shouldSetProperAttributableIdIfFound() or just modify the expected result to match the actual result
2. on PersonAttributeTypeResourceWithConcept1_9Test() please look into validateFullRepresentation() because inspite of the correct approach to cast an instance of SimpleObject to an org.openmrs.Concept it still throws a ClassCastException
3. i have commented out both of this test cases as i wait for your thoughts on these two issues

Cc. @dkayiwa @mks-d

I've fixed the tests so they work (according to me).

I've fixed the tests so they work (according to me).

thank you for this fix
it also makes sense to for

I've fixed the tests so they work (according to me).

it makes sense with me too

let me these changes update my repository and send my last commit
iam gonna make a new pr if you don't mind @ibacher

hello @ibacher
i have created a new pr

see #624

please look into it i think its ready for mergeing